1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

YJIT: Fix a compilation warning with release build (#7092)

Browse Source 
warning: unused variable: `start_addr`
   --> ../yjit/src/asm/mod.rs:359:39
    |
359 |     pub fn remove_comments(&mut self, start_addr: CodePtr, end_addr: CodePtr) {
    |                                       ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_start_addr`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: unused variable: `end_addr`
   --> ../yjit/src/asm/mod.rs:359:60
    |
359 |     pub fn remove_comments(&mut self, start_addr: CodePtr, end_addr: CodePtr) {
    |
This commit is contained in:
Takashi Kokubun 2023-01-10 08:00:25 -08:00 committed by GitHub
parent be1db1ca5c
commit 6a585dbd5a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
Notes: git 2023-01-10 16:00:46 +00:00 
Merged-By: maximecb <maximecb@ruby-lang.org>
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
yjit/src/asm/mod.rs
View File

@ -356,12 +356,15 @@ impl CodeBlock {
self.asm_comments.get(&pos)
}
#[cfg(feature = "disasm")]
pub fn remove_comments(&mut self, start_addr: CodePtr, end_addr: CodePtr) {
#[cfg(feature = "disasm")]
for addr in start_addr.into_usize()..end_addr.into_usize() {
self.asm_comments.remove(&addr);
}
}
#[cfg(not(feature = "disasm"))]
#[inline]
pub fn remove_comments(&mut self, _: CodePtr, _: CodePtr) {}
pub fn clear_comments(&mut self) {
#[cfg(feature = "disasm")]