[PATCH] D120260: [BOLT][NFC] Fix undefined behavior in encodeAnnotationImm
    Vladislav Khmelevsky via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Feb 23 13:10:28 PST 2022
    
    
  
yota9 added inline comments.
================
Comment at: bolt/unittests/Core/MCPlusBuilder.cpp:113
+TEST_P(MCPlusBuilderTester, Annotation) {
+  if (GetParam() != Triple::x86_64)
+    GTEST_SKIP();
----------------
Amir wrote:
> maksfb wrote:
> > Any reason for x86_64-only?
> `createCall` is not defined in AArch64MCPlusBuilder
Sorry, but why not just use BC->MIB->addAnnotation directly? 
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120260/new/
https://reviews.llvm.org/D120260
    
    
More information about the llvm-commits
mailing list