[PATCH] D70595: [TargetLowering] Allow constants with multiple uses
    David Stuttard via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Nov 22 04:25:25 PST 2019
    
    
  
dstuttard marked 2 inline comments as done.
dstuttard added inline comments.
================
Comment at: llvm/test/CodeGen/AMDGPU/const-multiuse-tl.ll:1
+; ModuleID = 'foo-new.bc'
+source_filename = "foo.ll"
----------------
lebedev.ri wrote:
> This test doesn't actually check anything.
Yes, spotted it just after I uploaded. It does now. It's really only testing that it doesn't assert.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70595/new/
https://reviews.llvm.org/D70595
    
    
More information about the llvm-commits
mailing list