[PATCH] D70595: [TargetLowering] Allow constants with multiple uses

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added reviewers: lebedev.ri, spatel.
RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/const-multiuse-tl.ll:1
+; ModuleID = 'foo-new.bc'
+source_filename = "foo.ll"
----------------
dstuttard wrote:
> 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.
Is this reduced as much as possible (both in terms of instructions and metadata)?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70595/new/

https://reviews.llvm.org/D70595